Skip to content
New issue

Have a question about this project? Sign up for a free GitHub account to open an issue and contact its maintainers and the community.

By clicking “Sign up for GitHub”, you agree to our terms of service and privacy statement. We’ll occasionally send you account related emails.

Already on GitHub? Sign in to your account

feat: add Sequencer for spring boot #41

Open
wants to merge 3 commits into
base: main
Choose a base branch
from
Open

Conversation

fft0518
Copy link
Contributor

@fft0518 fft0518 commented Sep 25, 2022

This PR is to add Sequencer annotation to Spring Boot and is one of the OSPP projects

mosn/layotto#626

@ZLBer
Copy link
Contributor

ZLBer commented Sep 27, 2022

@fft0518 resolve conversation
image

@ZLBer
Copy link
Contributor

ZLBer commented Sep 27, 2022

@fft0518 did you test the examples/src/main/java/io/mosn/layotto/examples/pubsub/springboot/SequencerMethod.java in real environment?

@fft0518
Copy link
Contributor Author

fft0518 commented Sep 27, 2022

@fft0518 did you test the examples/src/main/java/io/mosn/layotto/examples/pubsub/springboot/SequencerMethod.java in real environment?

Yes.

@fft0518
Copy link
Contributor Author

fft0518 commented Sep 27, 2022

@ZLBer said the pr is no problem, @seeflood please help to review it. Thanks.

@ZLBer
Copy link
Contributor

ZLBer commented Sep 29, 2022

@seeflood take a look at this pr

Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Labels
None yet
Projects
None yet
Development

Successfully merging this pull request may close these issues.

[OSPP 2022] improve layotto-java-sdk and layotto-spring-boot
2 participants